Skip to content

Feature Flags screenshot refresh [DOCS-15099]#38310

Merged
joepeeples merged 13 commits into
masterfrom
joepeeples/15108-flags-screenshot-refresh
Jul 22, 2026
Merged

Feature Flags screenshot refresh [DOCS-15099]#38310
joepeeples merged 13 commits into
masterfrom
joepeeples/15108-flags-screenshot-refresh

Conversation

@joepeeples

@joepeeples joepeeples commented Jul 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

  • Refreshes outdated images for Feature Flags docs.
  • Touches up some related text in limited contexts (full-scale content revision is outside the scope of the PR; we can address stale content in a separate PR).
  • Relocates a few image files that were previously located in the getting_started subfolder, but they weren't part of the Getting Started page.
  • Deletes an orphan image file (probably used in the past, no longer referenced in the docs for any language).

Merge readiness

  • Ready for merge

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

AI assistance

Claude for planning, cataloging images involved.

Additional notes

@github-actions github-actions Bot added the Images Images are added/removed with this PR label Jul 20, 2026
@github-actions

github-actions Bot commented Jul 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@joepeeples
update and relocate images
87f28f3 
images were stored under `getting_started/feature_flags` but weren't used on that page. originals kept there for translation process. pages updated:
- Environments
- Targeting Rules and Filters
- Traffic Splitting and Randomization
@joepeeples
update Flag History
0e55d14
@joepeeples
add Stale Flags page to nav (previously wasn't included)
ee0e426
@github-actions github-actions Bot added the Architecture Everything related to the Doc backend label Jul 21, 2026
@joepeeples
joepeeples marked this pull request as ready for review July 21, 2026 18:22
@joepeeples
joepeeples requested a review from a team as a code owner July 21, 2026 18:22
@jeff-morgan-dd jeff-morgan-dd self-assigned this Jul 21, 2026
jeff-morgan-dd
jeff-morgan-dd approved these changes Jul 21, 2026
View reviewed changes

@jeff-morgan-dd jeff-morgan-dd left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@annabfenske

annabfenske commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Overall lgtm, a few nits:

Looks like we have a stale screenshot of the targeting rule canvas in this section:
https://docs-staging.datadoghq.com/joepeeples/15108-flags-screenshot-refresh/feature_flags/concepts/flag_graphs/#targeting-rule-evaluation-counts

Let's avoid showing empty states on our graphs like the one shown in this screenshot (you can set the flag's distribution channels to client only to hide the server evaluations graph):
https://docs-staging.datadoghq.com/joepeeples/15108-flags-screenshot-refresh/getting_started/feature_flags/?tab=javascriptbrowser#step-5-monitor-your-rollout

@joepeeples

joepeeples commented Jul 22, 2026

Copy link
Copy Markdown
Contributor Author

Looks like we have a stale screenshot of the targeting rule canvas in this section: https://docs-staging.datadoghq.com/joepeeples/15108-flags-screenshot-refresh/feature_flags/concepts/flag_graphs/#targeting-rule-evaluation-counts

Let's avoid showing empty states on our graphs like the one shown in this screenshot (you can set the flag's distribution channels to client only to hide the server evaluations graph): https://docs-staging.datadoghq.com/joepeeples/15108-flags-screenshot-refresh/getting_started/feature_flags/?tab=javascriptbrowser#step-5-monitor-your-rollout

@annabfenske Addressed in 0f0cfd6. For the targeting rule canvas, I omitted the added text, as it'll be easier for translating the docs into other languages.

@joepeeples
joepeeples merged commit 0d6aa9d into master Jul 22, 2026
18 checks passed
@joepeeples
joepeeples deleted the joepeeples/15108-flags-screenshot-refresh branch July 22, 2026 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jeff-morgan-dd jeff-morgan-dd jeff-morgan-dd approved these changes

Assignees

@jeff-morgan-dd jeff-morgan-dd

Labels

Architecture Everything related to the Doc backend Images Images are added/removed with this PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@joepeeples @annabfenske @jeff-morgan-dd